stm32mp1: do not include platform header files directly in drivers

Instead, only platform_def.h is included.
The required files to be included are added in stm32mp1_def.h.

Signed-off-by: Yann Gautier <[email protected]>

diff --git a/drivers/st/clk/stm32mp1_clk.c b/drivers/st/clk/stm32mp1_clk.c
index 7a8898672dd5b540f7c48b2d76853b0b3164c4e7..e4cc9b9e8c9d68616fe0a12fdbd96927ed078b1a 100644 (file)
--- a/drivers/st/clk/stm32mp1_clk.c
+++ b/drivers/st/clk/stm32mp1_clk.c
@@ -11,6 +11,8 @@
 
 #include <libfdt.h>
 
+#include <platform_def.h>
+
 #include <arch.h>
 #include <arch_helpers.h>
 #include <common/debug.h>
@@ -25,9 +27,6 @@
 #include <lib/utils_def.h>
 #include <plat/common/platform.h>
 
-#include <stm32mp1_dt.h>
-#include <stm32mp1_private.h>
-
 #define MAX_HSI_HZ     64000000
 
 #define TIMEOUT_200MS  (plat_get_syscnt_freq2() / 5U)

diff --git a/drivers/st/clk/stm32mp1_clkfunc.c b/drivers/st/clk/stm32mp1_clkfunc.c
index 179191945f9ee49e3c99dde13ec8e5c729a7823b..1d92271063c8f4a054845c2979e555675d9a7fd0 100644 (file)
--- a/drivers/st/clk/stm32mp1_clkfunc.c
+++ b/drivers/st/clk/stm32mp1_clkfunc.c
@@ -8,12 +8,12 @@
 
 #include <libfdt.h>
 
+#include <platform_def.h>
+
 #include <drivers/st/stm32mp1_clk.h>
 #include <drivers/st/stm32mp1_clkfunc.h>
 #include <dt-bindings/clock/stm32mp1-clksrc.h>
 
-#include <stm32mp1_dt.h>
-
 #define DT_RCC_NODE_NAME       "rcc@50000000"
 #define DT_RCC_CLK_COMPAT      "st,stm32mp1-rcc"
 #define DT_RCC_COMPAT          "syscon"

diff --git a/drivers/st/ddr/stm32mp1_ddr.c b/drivers/st/ddr/stm32mp1_ddr.c
index a2b0e9ec05b48a57d0a57a99ba04dd64dc720f94..aca04504f6ca6f9ce5cb54a5faedf091677954fa 100644 (file)
--- a/drivers/st/ddr/stm32mp1_ddr.c
+++ b/drivers/st/ddr/stm32mp1_ddr.c
@@ -6,6 +6,8 @@
 
 #include <stddef.h>
 
+#include <platform_def.h>
+
 #include <arch.h>
 #include <arch_helpers.h>
 #include <common/debug.h>
@@ -21,9 +23,6 @@
 #include <lib/mmio.h>
 #include <plat/common/platform.h>
 
-#include <stm32mp1_def.h>
-#include <stm32mp1_dt.h>
-
 struct reg_desc {
        const char *name;
        uint16_t offset;        /* Offset for base address */

diff --git a/drivers/st/ddr/stm32mp1_ram.c b/drivers/st/ddr/stm32mp1_ram.c
index 1c821791799c75eb1240a024ed2026cc77cff00b..127b6c7c1c85ea08107a2c54ccd2d876f993bb6e 100644 (file)
--- a/drivers/st/ddr/stm32mp1_ram.c
+++ b/drivers/st/ddr/stm32mp1_ram.c
@@ -20,10 +20,6 @@
 #include <dt-bindings/clock/stm32mp1-clks.h>
 #include <lib/mmio.h>
 
-#include <boot_api.h>
-#include <stm32mp1_dt.h>
-#include <stm32mp1_private.h>
-
 #define DDR_PATTERN    0xAAAAAAAAU
 #define DDR_ANTIPATTERN        0x55555555U
 

diff --git a/drivers/st/io/io_stm32image.c b/drivers/st/io/io_stm32image.c
index ba612e2e9333a08539bea99944bc91036e745212..0164a2d43da4688fe9d647aa88fdc32e7caedc59 100644 (file)
--- a/drivers/st/io/io_stm32image.c
+++ b/drivers/st/io/io_stm32image.c
@@ -18,8 +18,6 @@
 #include <lib/utils.h>
 #include <plat/common/platform.h>
 
-#include <boot_api.h>
-
 static uintptr_t backend_dev_handle;
 static uintptr_t backend_image_spec;
 static uint32_t *stm32_img;

diff --git a/drivers/st/mmc/stm32_sdmmc2.c b/drivers/st/mmc/stm32_sdmmc2.c
index 787241d597486c98e1664bde7846639cd9b8aee5..05f5ae17c9d802da9f0ad0edbb1b7cc62585992b 100644 (file)
--- a/drivers/st/mmc/stm32_sdmmc2.c
+++ b/drivers/st/mmc/stm32_sdmmc2.c
@@ -8,6 +8,10 @@
 #include <errno.h>
 #include <string.h>
 
+#include <libfdt.h>
+
+#include <platform_def.h>
+
 #include <arch.h>
 #include <arch_helpers.h>
 #include <common/debug.h>
@@ -19,13 +23,10 @@
 #include <drivers/st/stm32mp1_reset.h>
 #include <dt-bindings/clock/stm32mp1-clks.h>
 #include <dt-bindings/reset/stm32mp1-resets.h>
-#include <libfdt.h>
 #include <lib/mmio.h>
 #include <lib/utils.h>
 #include <plat/common/platform.h>
 
-#include <stm32mp1_dt.h>
-
 /* Registers offsets */
 #define SDMMC_POWER                    0x00U
 #define SDMMC_CLKCR                    0x04U

diff --git a/drivers/st/pmic/stm32mp1_pmic.c b/drivers/st/pmic/stm32mp1_pmic.c
index ee4402762f010fdd6a62215c6713d20b89d6f6b8..c5bdfc07d3a1457dc570c3948e451ad5410ae357 100644 (file)
--- a/drivers/st/pmic/stm32mp1_pmic.c
+++ b/drivers/st/pmic/stm32mp1_pmic.c
@@ -20,9 +20,6 @@
 #include <lib/mmio.h>
 #include <lib/utils_def.h>
 
-#include <stm32mp1_def.h>
-#include <stm32mp1_dt.h>
-
 /* I2C Timing hard-coded value, for I2C clock source is HSI at 64MHz */
 #define I2C_TIMING                     0x10D07DB5
 

diff --git a/include/drivers/st/stm32mp1_pmic.h b/include/drivers/st/stm32mp1_pmic.h
index 256e340bec109a824bd4f283cefb6b9bab6be53e..75f8e61f7206c0a206e94c015ae1d763ed6d6350 100644 (file)
--- a/include/drivers/st/stm32mp1_pmic.h
+++ b/include/drivers/st/stm32mp1_pmic.h
@@ -9,7 +9,7 @@
 
 #include <stdbool.h>
 
-#include <stm32mp1_def.h>
+#include <platform_def.h>
 
 bool dt_check_pmic(void);
 int dt_pmic_enable_boot_on_regulators(void);

diff --git a/plat/st/stm32mp1/platform.mk b/plat/st/stm32mp1/platform.mk
index 8bb7a80e1a6fa8ac58322ca64ce2cd98f46a4c08..60852c6b4f965e940a6ab81f99584ba182f0c110 100644 (file)
--- a/plat/st/stm32mp1/platform.mk
+++ b/plat/st/stm32mp1/platform.mk
@@ -22,8 +22,6 @@ PLAT_PARTITION_MAX_ENTRIES    :=      $(shell echo $$(($(STM32_TF_A_COPIES) + 1)))
 $(eval $(call add_define,PLAT_PARTITION_MAX_ENTRIES))
 
 PLAT_INCLUDES          :=      -Iplat/st/stm32mp1/include/
-PLAT_INCLUDES          +=      -Iplat/st/stm32mp1/
-PLAT_INCLUDES          +=      -Iinclude/common/tbbr
 
 # Device tree
 STM32_DTB_FILE_NAME    ?=      stm32mp157c-ev1.dtb

diff --git a/plat/st/stm32mp1/stm32mp1_def.h b/plat/st/stm32mp1/stm32mp1_def.h
index fb21acceca58b2ee9d9f38a9d0fe64d701093578..15f0432b394a3a1dca91c1b365362171056955cb 100644 (file)
--- a/plat/st/stm32mp1/stm32mp1_def.h
+++ b/plat/st/stm32mp1/stm32mp1_def.h
@@ -11,6 +11,12 @@
 #include <lib/utils_def.h>
 #include <lib/xlat_tables/xlat_tables_defs.h>
 
+#ifndef __ASSEMBLY__
+#include <boot_api.h>
+#include <stm32mp1_dt.h>
+#include <stm32mp1_private.h>
+#endif
+
 /*******************************************************************************
  * STM32MP1 memory map related constants
  ******************************************************************************/